Skip to content

feat: add custom search bar for people table - #1477

Merged
ellen-wright merged 51 commits into
developfrom
1546-add-custom-search-bar-for-people-table
Jun 24, 2026
Merged

feat: add custom search bar for people table#1477
ellen-wright merged 51 commits into
developfrom
1546-add-custom-search-bar-for-people-table

Conversation

@ellen-wright

Copy link
Copy Markdown
Contributor

Description

refs: UserOfficeProject/issue-tracker#1546

Replaces the built in search bar for the material table with a custom search bar that allows the search to only execute on 'Enter'.

This fixes an issue for performance with the People Table search, by waiting to execute the search until the user hits 'Enter' it reduces the number of times the query is ran and allows enough characters of a name to be typed and therefore only a small number of results are returned.

Changes

Creates a new search bar that only executes on 'Enter'
I tried to make the new search look as similar as possible to the built in material table search.

  • I have added tests to cover my changes.
  • All relevant doc has been updated

@ellen-wright
ellen-wright requested a review from a team as a code owner April 13, 2026 13:44
@ellen-wright
ellen-wright requested review from ACLay and removed request for a team April 13, 2026 13:44

@yoganandaness yoganandaness left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me and approving this. Btw, it is worth adding some e2e.

@ellen-wright
ellen-wright marked this pull request as draft June 8, 2026 12:01
ellen-wright and others added 23 commits June 10, 2026 11:16
@ellen-wright
ellen-wright marked this pull request as ready for review June 24, 2026 15:25
@ellen-wright
ellen-wright merged commit 4d50b11 into develop Jun 24, 2026
35 of 36 checks passed
@ellen-wright
ellen-wright deleted the 1546-add-custom-search-bar-for-people-table branch June 24, 2026 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants